From: Oscar Sommerer <unlined-fielder0y@icloud.com>
Date: Sat, 31 May 2025 18:47:03 +0000 (+0200)
Subject: workspaceSwitcherPopup: Show on all displays
X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~124^2^2~42
X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=b0f0aace6b46212a6a0648337570c74be4057c2f;p=gnome-shell.git

workspaceSwitcherPopup: Show on all displays

Show the workspaceSwitcherPopup OSD on all displays if
`workspaces-only-on-primary` is false.

Closes https://gitlab.gnome.org/GNOME/gnome-shell/-/issues/8105

Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/3734>
---

diff --git a/js/ui/windowManager.js b/js/ui/windowManager.js
index b74d878e1..05aaeb240 100644
--- a/js/ui/windowManager.js
+++ b/js/ui/windowManager.js
@@ -1695,7 +1695,7 @@ export class WindowManager {
         if (!constructor)
             return;
 
-        /* prevent a corner case where both popups show up at once */
+        /* prevent a corner case where both tab and workspace switcher popups show up at once */
         if (this._workspaceSwitcherPopup != null)
             this._workspaceSwitcherPopup.destroy();
 
diff --git a/js/ui/workspaceSwitcherPopup.js b/js/ui/workspaceSwitcherPopup.js
index bf383d33a..a1b66327d 100644
--- a/js/ui/workspaceSwitcherPopup.js
+++ b/js/ui/workspaceSwitcherPopup.js
@@ -1,6 +1,7 @@
 import Clutter from 'gi://Clutter';
 import GLib from 'gi://GLib';
 import GObject from 'gi://GObject';
+import Meta from 'gi://Meta';
 import St from 'gi://St';
 
 import * as Layout from './layout.js';
@@ -9,44 +10,26 @@ import * as Main from './main.js';
 const ANIMATION_TIME = 100;
 const DISPLAY_TIMEOUT = 600;
 
-
-export const WorkspaceSwitcherPopup = GObject.registerClass(
-class WorkspaceSwitcherPopup extends Clutter.Actor {
-    _init() {
-        super._init({
+export const MonitorWorkspaceSwitcherPopup = GObject.registerClass(
+class MonitorWorkspaceSwitcherPopup extends Clutter.Actor {
+    constructor(constraint) {
+        super({
             offscreen_redirect: Clutter.OffscreenRedirect.ALWAYS,
             x_expand: true,
             y_expand: true,
             x_align: Clutter.ActorAlign.CENTER,
             y_align: Clutter.ActorAlign.END,
         });
-
-        const constraint = new Layout.MonitorConstraint({primary: true});
         this.add_constraint(constraint);
 
-        Main.uiGroup.add_child(this);
-
-        this._timeoutId = 0;
-
         this._list = new St.BoxLayout({
             style_class: 'workspace-switcher',
         });
         this.add_child(this._list);
-
-        this._redisplay();
-
-        this.hide();
-
-        let workspaceManager = global.workspace_manager;
-        workspaceManager.connectObject(
-            'workspace-added', this._redisplay.bind(this),
-            'workspace-removed', this._redisplay.bind(this), this);
-
-        this.connect('destroy', this._onDestroy.bind(this));
     }
 
-    _redisplay() {
-        let workspaceManager = global.workspace_manager;
+    redisplay(activeWorkspaceIndex) {
+        const workspaceManager = global.workspace_manager;
 
         this._list.destroy_all_children();
 
@@ -55,22 +38,58 @@ class WorkspaceSwitcherPopup extends Clutter.Actor {
                 style_class: 'ws-switcher-indicator',
             });
 
-            if (i === this._activeWorkspaceIndex)
+            if (i === activeWorkspaceIndex)
                 indicator.add_style_pseudo_class('active');
 
             this._list.add_child(indicator);
         }
     }
+});
+
+export const WorkspaceSwitcherPopup = GObject.registerClass(
+class WorkspaceSwitcherPopup extends Clutter.Actor {
+    constructor() {
+        super();
+
+        this._timeoutId = 0;
+
+        Main.uiGroup.add_child(this);
+
+        this.hide();
+
+        if (Meta.prefs_get_workspaces_only_on_primary()) {
+            const constraint = new Layout.MonitorConstraint({primary: true});
+            this.add_child(new MonitorWorkspaceSwitcherPopup(constraint));
+        } else {
+            const monitors = Main.layoutManager.monitors;
+            monitors.forEach((_, index) => {
+                const constraint = new Layout.MonitorConstraint({index});
+                this.add_child(new MonitorWorkspaceSwitcherPopup(constraint));
+            });
+        }
+
+        const workspaceManager = global.workspace_manager;
+        workspaceManager.connectObject(
+            'workspace-added', this._redisplayAllPopups.bind(this),
+            'workspace-removed', this._redisplayAllPopups.bind(this), this);
+        this.connect('destroy', this._onDestroy.bind(this));
+    }
+
+    _redisplayAllPopups() {
+        for (const popup of this)
+            popup.redisplay(this._activeWorkspaceIndex);
+    }
 
     display(activeWorkspaceIndex) {
         this._activeWorkspaceIndex = activeWorkspaceIndex;
 
-        this._redisplay();
         if (this._timeoutId !== 0)
             GLib.source_remove(this._timeoutId);
         this._timeoutId = GLib.timeout_add(GLib.PRIORITY_DEFAULT, DISPLAY_TIMEOUT, this._onTimeout.bind(this));
         GLib.Source.set_name_by_id(this._timeoutId, '[gnome-shell] this._onTimeout');
 
+        this._redisplayAllPopups();
+
         const duration = this.visible ? 0 : ANIMATION_TIME;
         this.show();
         this.opacity = 0;